Skip to content

Simplify Lighthouse CI workflow by removing GitHub CLI setup#2

Merged
dpup merged 1 commit into
mainfrom
claude/fix-lighthouse-auth-PF3dj
Feb 24, 2026
Merged

Simplify Lighthouse CI workflow by removing GitHub CLI setup#2
dpup merged 1 commit into
mainfrom
claude/fix-lighthouse-auth-PF3dj

Conversation

@dpup
Copy link
Copy Markdown
Contributor

@dpup dpup commented Feb 24, 2026

Summary

Simplified the Lighthouse CI GitHub Actions workflow by removing unnecessary GitHub CLI installation and authentication steps.

Key Changes

  • Removed the "Setup GitHub CLI" step that installed the GitHub CLI tool and its dependencies
  • Removed the "Authenticate GitHub CLI" step that authenticated with GitHub using a token
  • Removed the LHCI_GITHUB_APP_TOKEN secret from the Lighthouse CI environment variables
  • Kept the GH_TOKEN environment variable for the Lighthouse CI step

Details

The GitHub CLI setup and authentication were not being used by the bun run test:lighthouse command, making them unnecessary overhead in the workflow. This change reduces workflow complexity and execution time by eliminating redundant installation and authentication steps while maintaining the required token access for Lighthouse CI operations.

https://claude.ai/code/session_01U1HYSJ2cRcy99YsBjfjqgd

Remove the "Setup GitHub CLI" and "Authenticate GitHub CLI" steps that
are not used by the working deploy and validate workflows. Remove the
LHCI_GITHUB_APP_TOKEN env var which references a non-existent secret.

The gh CLI is pre-installed on ubuntu-latest and automatically uses the
GH_TOKEN env var — no explicit gh auth login is needed, matching the
pattern used by deploy.yml and validate.yml.

https://claude.ai/code/session_01U1HYSJ2cRcy99YsBjfjqgd
@dpup dpup merged commit f89ec75 into main Feb 24, 2026
1 of 2 checks passed
@dpup dpup deleted the claude/fix-lighthouse-auth-PF3dj branch February 24, 2026 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants